Skip to content

Conversation

@karknu
Copy link
Contributor

@karknu karknu commented Jun 24, 2025

Description

Lower severity for AcquireConnectionError from Error to Info.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. These may include:
    • golden tests
    • property tests
    • roundtrip tests
    • integration tests
      See Runnings tests for more details
  • Any changes are noted in the CHANGELOG.md for affected package
  • The version bounds in .cabal files are updated
  • CI passes. See note on CI. The following CI checks are required:
    • Code is linted with hlint. See .github/workflows/check-hlint.yml to get the hlint version
    • Code is formatted with stylish-haskell. See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
    • Code builds on Linux, MacOS and Windows for ghc-9.6 and ghc-9.12
  • Self-reviewed the diff

Note on CI

If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG node developers to do this
for you.

@mgmeier
Copy link
Contributor

mgmeier commented Jun 24, 2025

That change targets the old tracing system. Could you please make sure the new tracing system stays in sync?

(module cardano-node/src/Cardano/Node/Tracing/Tracers/P2P.hs, instance MetaTrace (PeerSelectionActionsTrace SockAddr lAddr definition of severityFor) - Thanks.

Lower severity for PeerStatusChangeFailure, PeerMonitoringError and
AcquireConnectionError.

Communication with the remote peer may timeout or error.
But this doesn't represent a "cardano-node" error.
@karknu karknu force-pushed the karknu/acquirecon_sev branch from 5cac46f to fffbd95 Compare June 24, 2025 12:13
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale label Aug 12, 2025
@github-actions
Copy link

This issue was closed because it has been stalled for 60 days with no activity.

@github-actions github-actions bot closed this Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants